Skip to content

Add Generic ConfigDB Validation via YANG Design Doc#1072

Merged
qiluo-msft merged 7 commits intosonic-net:masterfrom
isabelmsft:yang_cli_validation
Oct 12, 2022
Merged

Add Generic ConfigDB Validation via YANG Design Doc#1072
qiluo-msft merged 7 commits intosonic-net:masterfrom
isabelmsft:yang_cli_validation

Conversation

@isabelmsft
Copy link
Contributor

@isabelmsft isabelmsft commented Aug 25, 2022 •
edited
Loading

This PR contains the ConfigDB Validation via YANG HLD.

PR title (included in 202211 branch) state context
[sonic-utilities] Added CLI to enable/disable yang config validation in ConfigDB GitHub issue/pull request detail GitHub pull request check contexts
[sonic-buildimage] Add yang_config_validation to DEVICE_METADATA yang model GitHub issue/pull request detail GitHub pull request check contexts
[sonic-buildimage] Add yang_config_validation to minigraph.py GitHub issue/pull request detail GitHub pull request check contexts
[sonic-utilities] Added support for ConfigDB updates via YANG validation - including validation for portchannel, loopback, and vlan interfaces GitHub issue/pull request detail GitHub pull request check contexts
[sonic-utilities] Add YANG validation for portchannel member and mgmt interface GitHub issue/pull request detail GitHub pull request check contexts
[sonic-utilities] YANG validation for ConfigDB Updates: MIRROR_SESSION use case GitHub issue/pull request detail GitHub pull request check contexts
[sonic-utilities] YANG Validation for ConfigDB Updates: TACPLUS, TACPLUS_SERVER, AAA, VLAN_SUB_INTERFACE, NETWORKING_METADATA tables + decorated validated_mod_entry GitHub issue/pull request detail GitHub pull request check contexts
[sonic-utilities] YANG Validation for ConfigDB Updates: DEVICE_METADATA, SNMP, SNMP_COMMUNITY GitHub issue/pull request detail GitHub pull request check contexts
[sonic-utilities] YANG Validation for ConfigDB Updates: PORT_STORM_CONTROL, PORT_QOS_MAP, BUFFER_PROFILE, BUFFER_PG, BUFFER_QUEUE, BUFFER_POOL, FEATURE, DEFAULT_LOSSLESS_BUFFER_PARAMETER tables GitHub issue/pull request detail GitHub pull request check contexts
PR title (to be cherrypicked into 202211 branch) state context
[sonic-utilities] YANG Validation for ConfigDB Updates: WARM_RESTART, SFLOW_SESSION, SFLOW, VXLAN_TUNNEL, VXLAN_EVPN_NVO, VXLAN_TUNNEL_MAP, MGMT_VRF_CONFIG, CABLE_LENGTH, VRF tables GitHub issue/pull request detail GitHub pull request check contexts
[[sonic-utilities] YANG Validation for ConfigDB Updates: RADIUS_SERVER tables ](sonic-net/sonic-utilities#2604) GitHub issue/pull request detail GitHub pull request check contexts
[sonic-utilities] YANG Validation for ConfigDB Updates: CONSOLE_SWITCH, CONSOLE_PORT, KUBERNETES_MASTER, BREAKOUT_CFG, NTP_SERVER, SFLOW_COLLECTOR, MCLAG_DOMAIN, MCLAG_INTERFACE, MCLAG_UNIQUE_IP, MUX_CABLE, STATIC_NAT, STATIC_NAPT, NAT_POOL, NAT_BINDINGS, NAT_GLOBAL tables GitHub issue/pull request detail GitHub pull request check contexts
YANG Model GitHub Issue state
[yang] MIRROR_SESSION GitHub issue/pull request detail
[yang] VLAN_SUB_INTERFACE GitHub issue/pull request detail
[yang] RADIUS_SERVER, RADIUS missing YANG models GitHub issue/pull request detail
[yang] SNMP_AGENT_ADDRESS_CONFIG, SNMP_USER missing YANG models GitHub issue/pull request detail
[yang] SNMP GitHub issue/pull request detail
[yang] CHASSIS_MODULE GitHub issue/pull request detail
[yang] XCVRD_LOG GitHub issue/pull request detail

dgsudharsan
dgsudharsan requested changes Sep 28, 2022
View reviewed changes
@qiluo-msft
Copy link
Contributor

qiluo-msft commented Oct 4, 2022

@isabelmsft Please add all the related PRs table into HLD PR description. So in future, audience will have a great overview of the project status, and easily cherry-pick them to any other branches.

For example: #1024

qiluo-msft
qiluo-msft reviewed Oct 4, 2022
View reviewed changes
@isabelmsft
Copy link
Contributor Author

isabelmsft commented Oct 5, 2022

@isabelmsft Please add all the related PRs table into HLD PR description. So in future, audience will have a great overview of the project status, and easily cherry-pick them to any other branches.

For example: #1024

I have added the table for related PRs to this PR description

qiluo-msft
qiluo-msft previously approved these changes Oct 5, 2022
View reviewed changes
@qiluo-msft qiluo-msft merged commit a181a5a into sonic-net:master Oct 12, 2022
@oleksandrivantsiv oleksandrivantsiv mentioned this pull request Jun 8, 2023
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@qiluo-msft qiluo-msft qiluo-msft approved these changes

@dgsudharsan dgsudharsan dgsudharsan approved these changes

Assignees

No one assigned

Labels

Request for 202211 Branch

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

4 participants

@isabelmsft @qiluo-msft @dgsudharsan @zhangyanzhao